Skip to content

[Chore] Build downloadable VSIX artifacts for pull requests - #1314

Merged
taltas merged 1 commit into
mainfrom
feature/pr-vsix-artifacts-3jm1f5fmjn8i2
Aug 22, 2026
Merged

[Chore] Build downloadable VSIX artifacts for pull requests#1314
taltas merged 1 commit into
mainfrom
feature/pr-vsix-artifacts-3jm1f5fmjn8i2

Conversation

@zoomote

@zoomote zoomote Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

​Opened on behalf of @navedmerchant. View the task or mention @roomote for follow-up asks.

Related GitHub Issue

No issue was provided for this CI improvement.

Description

Adds a PR-only Build test VSIX job to the existing Code QA workflow. The job builds the required workspace packages, packages Zoo Code, and uploads the resulting VSIX as a seven-day GitHub Actions artifact named for the pull request.

This gives contributors and reviewers an installable build for testing fixes before merge. The job keeps read-only repository permissions, disables persisted checkout credentials, uses no secrets, and does not create GitHub Releases or publish to either extension marketplace.

Test Procedure

  1. Confirm the Build test VSIX check succeeds on this pull request.
  2. Open the completed workflow run and download zoo-code-vsix-pr-<PR number> from its Artifacts section.
  3. Extract the artifact and install the VSIX with VS Code's Extensions: Install from VSIX... command.

Locally verified with the exact workspace build and package commands; bin/zoo-code-3.78.0.vsix was produced. The full pnpm test suite passed with 7,538 tests, and the commit and push hooks also passed lint and type checking.

Pre-Submission Checklist

  • Issue Linked: This PR is linked to an approved GitHub Issue (see "Related GitHub Issue" above).
  • Scope: My changes are focused on the linked issue (one major feature/fix per PR).
  • Self-Review: I have performed a thorough self-review of my code.
  • Testing: New and/or updated tests have been added to cover my changes (if applicable).
  • Visual Snapshot (UI changes only): If a user would notice this change at a glance (layout, theme tokens, brand elements, empty/error states), I've added or updated a *.visual.tsx snapshot in webview-ui/. See webview-ui/AGENTS.md -> "When a UI change needs a snapshot".
  • Documentation Impact: I have considered if my changes require documentation updates (see "Documentation Updates" section below).
  • Contribution Guidelines: I have read and agree to the Contributor Guidelines.

Visual Snapshots

Not applicable; this is a GitHub Actions workflow change.

Videos (interaction / animation only)

Not applicable.

Documentation Updates

  • No documentation updates are required.
  • Yes, documentation updates are required. (Please describe what needs to be updated or link to a PR in the docs repository).

Additional Notes

GitHub requires signed-in repository read access to download workflow artifacts. External or first-time contributor workflows may also wait for maintainer approval according to repository or organization policy; the workflow intentionally does not use pull_request_target to bypass that safety control. A recent fork PR successfully uploaded an artifact through this same Code QA workflow and pinned upload action.

Get in Touch

N/A

@codecov

codecov Bot commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@navedmerchant
navedmerchant marked this pull request as ready for review August 21, 2026 06:48
@github-actions github-actions Bot added the awaiting-review PR changes are ready and waiting for maintainer re-review label Aug 21, 2026
@taltas
taltas added this pull request to the merge queue Aug 22, 2026
Merged via the queue into main with commit d28e4a1 Aug 22, 2026
26 checks passed
@taltas
taltas deleted the feature/pr-vsix-artifacts-3jm1f5fmjn8i2 branch August 22, 2026 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-review PR changes are ready and waiting for maintainer re-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants